feat(sdk): add reviewed release notes and align Go timeouts - #5
Merged
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
GitHub Releases now publish reviewed notes from the matching
CHANGELOG.mdversion instead of relying on an empty release description. Forward and Managed clients also bound the default wait for response headers to 10 minutes, matching the pinned Anthropic Go SDK baseline (v1.74.0).Unreleasedand update README/API documentation.Why
Each release needs a version-specific explanation that reviewers can approve. A server that accepts a connection but never sends response headers should also have a default waiting limit without cutting off an established event stream.
Verification
go test ./...go test -tags live -run 'Cleanup.*Offline$' ./forward ./managedgo vet ./..., Go formatting checks, andmake build(including examples).Account-backed integration tests were not run; the changed transport behavior is verified with local HTTP servers and offline tests. GitHub release publication itself was not executed locally.
Impact
The default response-header wait changes from unbounded to 10 minutes. Applications needing different limits can provide an HTTP client; whole-call and per-attempt deadlines remain independently configurable. API resource contracts and fixtures do not change. No credentials, live environment files, or test output are included.
Related SDK changes: Python #7, TypeScript #7.